Export all media files per item instead of only the first#362
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
apodacaduron
left a comment
apodacaduron
left a comment
There was a problem hiding this comment.
Tested using local filesystem it does bring back all related files now
2 participants
Fixes BYNT-1727.
generate_export_mediacopied onlyitem.medias[0], so bulletins (and actors) with multiple media attachments exported incomplete data. Now all media files linked to each exported item are included, for both local filesystem and S3 storage. The per-file throttle sleep moved inside the media loop so S3 download pacing stays consistent.Media filenames are UUID-based so there are no collision concerns in the export folder.